Skip to content

Conversation

@pyansys-ci-bot
Copy link
Collaborator

An update of generated code has been triggered either manually or by an update in the dpf-standalone repository.

@pyansys-ci-bot pyansys-ci-bot requested a review from a team as a code owner November 4, 2025 16:56
@pyansys-ci-bot pyansys-ci-bot added the server-sync DO NOT USE, Related to automatic synchronization with the server label Nov 4, 2025
@@ -0,0 +1,950 @@
"""
markdown_latex_example
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PProfizi Will be interesting to see how is this one rendered in the pydpf-core doc!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PProfizi I'd say it is good at 80%, with some issues like this one:
image
Still in my opinion not a blocker to merge. We can focus on this in the future. Thanks

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @rafacanton thanks yeah the point is to fix this going forward, but this is not stopping the update.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually we can already see what the issue is, and seeing this was exactly the point:
our toolchain here supports dollar delimiters but not bracket/parentheses ones.
You can see the lines are duplicated, the first renders correctly, the second does not.
I'll also check the python docstring rendering in API reference.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can then also check how it renders in the Framework doc, and in Mechanical... etc.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

Some tests with 'continue-on-error: true' have failed:

  • PyDPF-Post API tests on ubuntu-latest

Created by continue-on-error-comment

@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.41%. Comparing base (da20bde) to head (b1cc7a2).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2725   +/-   ##
=======================================
  Coverage   84.41%   84.41%           
=======================================
  Files          92       92           
  Lines       10900    10900           
=======================================
  Hits         9201     9201           
  Misses       1699     1699           

@rafacanton rafacanton merged commit 45dcabc into main Nov 5, 2025
45 checks passed
@rafacanton rafacanton deleted the maint/update_code_for_261_daily_on_main branch November 5, 2025 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

server-sync DO NOT USE, Related to automatic synchronization with the server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants